Skip to content

Refactor serialization#17

Merged
boivie merged 3 commits intowebrtc:mainfrom
boivie:refactor-serialization
Aug 22, 2025
Merged

Refactor serialization#17
boivie merged 3 commits intowebrtc:mainfrom
boivie:refactor-serialization

Conversation

@boivie
Copy link
Copy Markdown
Collaborator

@boivie boivie commented Aug 22, 2025

No description provided.

@boivie boivie requested a review from lndmrk August 22, 2025 10:53
boivie added 3 commits August 22, 2025 12:55
This is used by both ForwardTsn and IForwardTsn and is a sum type of
skipped streams for both of them. Move it to the parent module as it's
referenced by both.
This avoids doing manual offset handling and for loops, and instead
iterates on a chunked view of the data, which is easier to maintain.
This replaces the old offset handling and for loops with iterators
operating on a chunked view of the data.
@boivie boivie force-pushed the refactor-serialization branch from 6772b05 to 711f4d9 Compare August 22, 2025 10:55
Copy link
Copy Markdown
Collaborator

@lndmrk lndmrk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@boivie boivie merged commit e22fac7 into webrtc:main Aug 22, 2025
6 checks passed
@boivie boivie deleted the refactor-serialization branch January 18, 2026 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants